scripts: add support for IRONside SE update service #22065

hakonfam · 2025-04-25T10:11:04Z

Add a new west command for performing an update.

The update command can be invoked as:

depends on #22133

Ref: NCSDK-32168

NordicBuilder · 2025-04-25T10:15:03Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 43

Inputs:

Sources:

sdk-nrf: PR head: d451bd231d4071c9989d7f78f6ae810db8a8dbc4

more details

sdk-nrf:

PR head: d451bd231d4071c9989d7f78f6ae810db8a8dbc4
merge base: 995c96c667fe3a8e494b393dde9058ad225057e1
target head (main): febd0cf20f3023b4ea44d603919e4f6e97c456df
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (4)

CODEOWNERS
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
scripts
│  ├── west-commands.yml
│  ├── west_commands
│  │  │ ncs_ironside_se_update.py

Outputs:

Toolchain

Version: 1d2c5d0ab5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:1d2c5d0ab5_8bf7ca4353

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 7
✅ Integration tests
- ✅ test-fw-nrfconnect-boot
- ✅ test-sdk-mcuboot

Disabled integration tests

- desktop52_verification
- doc-internal
- test_ble_nrf_config
- test-fw-nrfconnect-apps
- test-fw-nrfconnect-ble_mesh
- test-fw-nrfconnect-ble_samples
- test-fw-nrfconnect-chip
- test-fw-nrfconnect-fem
- test-fw-nrfconnect-nfc
- test-fw-nrfconnect-nrf-iot_cloud
- test-fw-nrfconnect-nrf-iot_libmodem-nrf
- test-fw-nrfconnect-nrf-iot_lwm2m
- test-fw-nrfconnect-nrf-iot_mosh
- test-fw-nrfconnect-nrf-iot_positioning
- test-fw-nrfconnect-nrf-iot_samples
- test-fw-nrfconnect-nrf-iot_serial_lte_modem
- test-fw-nrfconnect-nrf-iot_thingy91
- test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
- test-fw-nrfconnect-nrf_crypto
- test-fw-nrfconnect-proprietary_esb
- test-fw-nrfconnect-ps
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread
- test-low-level
- test-sdk-audio
- test-sdk-find-my
- test-sdk-pmic-samples
- test-sdk-wifi
- test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

scripts/west_commands/ncs_ironside_se_update.py

applications/ironside/se/update/README.rst

umapraseeda · 2025-04-29T06:56:41Z

Missing changelog entry?

applications/ironside/se/update/README.rst

include/tfm/ironside/se/ipc_service.h

lib/ironside/se/CMakeLists.txt

lib/ironside/se/ironside_se_update.c

include/ironside/se/update.h

include/tfm/ironside/se/ipc_service.h

lib/ironside/se/CMakeLists.txt

include/ironside/se/update.h

include/tfm/ironside/se/ipc_service.h

lib/ironside/se/CMakeLists.txt

peknis · 2025-04-30T06:17:13Z

Something is not right in the sample.yaml file as the doc cannot build the table.

Or then the new board is missing from https://github.com/nrfconnect/sdk-nrf/blob/main/doc/nrf/includes/sample_board_rows.txt

hakonfam · 2025-04-30T07:05:09Z

Something is not right in the sample.yaml file as the doc cannot build the table.

Or then the new board is missing from https://github.com/nrfconnect/sdk-nrf/blob/main/doc/nrf/includes/sample_board_rows.txt

Added an entry there now, lets see if that fixes it.

github-actions · 2025-04-30T07:29:21Z

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-22065/nrf/releases_and_maturity/releases/release-notes-changelog.html

umapraseeda · 2025-05-02T09:33:41Z

include/ironside/se/update.h

Missing library documentation? Should be documented under doc/nrf/libraries and added to the changelog.

Moved the implementation upstream.

hakonfam · 2025-05-05T04:40:05Z

The implementation has been moved here zephyrproject-rtos/zephyr#89333

All review feedback is contained in that PR.

Adds a new west command to install an IRONside SE update to an nRF54H20 device. Both the IRONside SE main firmware and recovery firmware can be updated. This command only works on nRF54H20 devices that already has a version of IRONside SE installed. The command is used by invoking: ``` west ncs-ironside-se-update ``` Ref: NCSDK-32168 Signed-off-by: Håkon Amundsen <[email protected]>

hakonfam requested review from a team as code owners April 25, 2025 10:11

github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 25, 2025

hakonfam force-pushed the update branch from da60092 to a86a49c Compare April 25, 2025 11:15

hakonfam requested a review from a team as a code owner April 25, 2025 11:15

github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 25, 2025

hakonfam force-pushed the update branch 9 times, most recently from f96df02 to c3f9c0e Compare April 28, 2025 08:09

hakonfam commented Apr 28, 2025

View reviewed changes

scripts/west_commands/ncs_ironside_se_update.py Outdated Show resolved Hide resolved

FrancescoSer requested changes Apr 28, 2025

View reviewed changes

hakonfam force-pushed the update branch from 682f3a5 to ed81b82 Compare April 28, 2025 09:56

hakonfam requested a review from FrancescoSer April 28, 2025 09:56

hakonfam force-pushed the update branch 2 times, most recently from daf4543 to 1e40308 Compare April 28, 2025 11:33

karstenkoenig reviewed Apr 28, 2025

View reviewed changes

applications/ironside/se/update/README.rst Outdated Show resolved Hide resolved

applications/ironside/se/update/README.rst Outdated Show resolved Hide resolved

hakonfam force-pushed the update branch from 1e40308 to 601322a Compare April 28, 2025 13:25

hakonfam requested a review from karstenkoenig April 29, 2025 04:30

hakonfam force-pushed the update branch from 601322a to a2c1a21 Compare April 29, 2025 07:57

FrancescoSer reviewed Apr 29, 2025

View reviewed changes

applications/ironside/se/update/README.rst Outdated Show resolved Hide resolved

NordicBuilder added the manifest-zephyr label Apr 29, 2025

karstenkoenig reviewed Apr 29, 2025

View reviewed changes

include/tfm/ironside/se/ipc_service.h Outdated Show resolved Hide resolved

lib/ironside/se/CMakeLists.txt Outdated Show resolved Hide resolved

lib/ironside/se/ironside_se_update.c Outdated Show resolved Hide resolved

lib/ironside/se/ironside_se_update.c Outdated Show resolved Hide resolved

57300 reviewed Apr 29, 2025

View reviewed changes

include/ironside/se/update.h Outdated Show resolved Hide resolved

include/ironside/se/update.h Outdated Show resolved Hide resolved

include/tfm/ironside/se/ipc_service.h Outdated Show resolved Hide resolved

lib/ironside/se/CMakeLists.txt Outdated Show resolved Hide resolved

jonathannilsen reviewed Apr 29, 2025

View reviewed changes

include/ironside/se/update.h Outdated Show resolved Hide resolved

include/ironside/se/update.h Outdated Show resolved Hide resolved

include/tfm/ironside/se/ipc_service.h Outdated Show resolved Hide resolved

lib/ironside/se/CMakeLists.txt Outdated Show resolved Hide resolved

hakonfam force-pushed the update branch from a8cc85b to 2aba623 Compare April 30, 2025 05:28

hakonfam requested a review from peknis April 30, 2025 05:29

hakonfam force-pushed the update branch from 2aba623 to de8a2b9 Compare April 30, 2025 05:47

hakonfam force-pushed the update branch from de8a2b9 to 052cd04 Compare April 30, 2025 06:46

jonathannilsen approved these changes Apr 30, 2025

View reviewed changes

hakonfam force-pushed the update branch from 052cd04 to 8cd5536 Compare April 30, 2025 07:04

umapraseeda reviewed May 2, 2025

View reviewed changes

hakonfam force-pushed the update branch from 8cd5536 to cf43379 Compare May 5, 2025 04:37

github-actions bot removed the manifest label May 5, 2025

NordicBuilder removed manifest-zephyr DNM labels May 5, 2025

hakonfam requested review from 57300 and karstenkoenig May 5, 2025 04:40

hakonfam changed the title ~~subsys: applications: add support for IRONside SE update service~~ scripts: add support for IRONside SE update service May 5, 2025

hakonfam requested review from divipillai and umapraseeda May 5, 2025 12:55

peknis approved these changes May 6, 2025

View reviewed changes

umapraseeda approved these changes May 6, 2025

View reviewed changes

divipillai approved these changes May 6, 2025

View reviewed changes

hakonfam force-pushed the update branch from cf43379 to d451bd2 Compare May 7, 2025 08:54

nordicjm merged commit 21a3bd3 into nrfconnect:main May 7, 2025
22 checks passed

scripts: add support for IRONside SE update service #22065

scripts: add support for IRONside SE update service #22065

Uh oh!

Conversation

hakonfam commented Apr 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

NordicBuilder commented Apr 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

umapraseeda commented Apr 29, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

peknis commented Apr 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hakonfam commented Apr 30, 2025

Uh oh!

github-actions bot commented Apr 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

umapraseeda May 2, 2025

Choose a reason for hiding this comment

Uh oh!

hakonfam May 5, 2025

Choose a reason for hiding this comment

Uh oh!

hakonfam commented May 5, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

hakonfam commented Apr 25, 2025 •

edited

Loading

NordicBuilder commented Apr 25, 2025 •

edited

Loading

peknis commented Apr 30, 2025 •

edited

Loading

github-actions bot commented Apr 30, 2025 •

edited

Loading